-
Notifications
You must be signed in to change notification settings - Fork 83
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: hide overflowing tree-toggle content #3525
Conversation
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works well on all supported browsers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like a clean and simple solution. Let’s hope we didn't miss any edge cases that this might trigger🤞(I didn't come up with any right away).
Hi @web-padawan , this commit cannot be picked to 22.0 by this bot, can you take a look and pick it manually? |
This ticket/PR has been released with Vaadin 23.0.2. |
This ticket/PR has been released with Vaadin 23.1.0.alpha1 and is also targeting the upcoming stable 23.1.0 version. |
Description
Added styles to the
slot
element as suggested by vaadin/flow-components#2429 (comment)Fixes vaadin/flow-components#2429
Type of change